-
Notifications
You must be signed in to change notification settings - Fork 599
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Service Catalog Datadog integration #8154
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Diana <[email protected]>
✅ Deploy Preview for kongdocs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Signed-off-by: Diana <[email protected]>
LGTM, thanks Diana! I'll merge the PR when we are ready for the Public Beta launch. FYI that there are a few UI polishes we'd like to address before going for an Internal Release (https://kongstrong.slack.com/archives/C068AHKGSPQ/p1731956817244099) , and so Public Beta won't happen for at least a week. Regardless, this PR is good to go from my perspective. |
Oh whoops, I accidentally clicked the "mark as ready for review" button. 😓 |
@miko-bautista No worries! I converted it back to a draft |
@miko-bautista Thanks for the review! I'd like to add a line after the integration instructions about selecting the region in Datadog (I think that's what happens here based on the brief) once I can test it in the internal release/beta. Do dashboards still need to be public for Konnect to ingest them? |
@cloudjumpercat I believe Datadog Dashboards do not need to be public for Service Catalog to ingest them, but I asked Miles a question as a comment on this document just to be sure: https://docs.google.com/document/d/14fNT_KPVEtgtIMlrI180jDwsHD4BaF7u5j4cqurbPKI/edit?disco=AAABY-yC8fs |
Description
Adding Datadog as a new Service Catalog integration.
DOCU-4109
Testing instructions
Preview link:
Checklist